Skip to content

Comments

ci: use Blacksmith runners in CI and integration workflows#136

Merged
benvinegar merged 1 commit intomainfrom
ci/blacksmith-runners
Feb 23, 2026
Merged

ci: use Blacksmith runners in CI and integration workflows#136
benvinegar merged 1 commit intomainfrom
ci/blacksmith-runners

Conversation

@benvinegar
Copy link
Contributor

@benvinegar benvinegar commented Feb 23, 2026

Summary

  • switch CI workflow jobs from ubuntu-latest to the Blacksmith runner class used in modem
  • switch integration workflow jobs to the same runner class
  • use a per-workflow YAML anchor (&default-runner) to avoid repeating the runner label

Testing

  • workflow-only change (no local runtime tests)

@greptile-apps
Copy link

greptile-apps bot commented Feb 23, 2026

Greptile Summary

Migrated CI and integration workflows from ubuntu-latest to Blacksmith runners (blacksmith-4vcpu-ubuntu-2404). The change uses YAML anchors (&default-runner and *default-runner) to avoid repetition across jobs, following the same pattern used in the modem repository.

Confidence Score: 5/5

  • Safe to merge - workflow configuration change with no runtime impact
  • This is a straightforward infrastructure change that only modifies the GitHub Actions runner platform. The YAML anchor syntax is correct and all jobs properly reference the anchor. No logic changes, no security implications, and no runtime behavior changes.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/ci.yml Replaced ubuntu-latest with Blacksmith runner using YAML anchor pattern across all jobs
.github/workflows/integration.yml Replaced ubuntu-latest with Blacksmith runner using YAML anchor pattern across all jobs

Last reviewed commit: 8cd27c9

@benvinegar benvinegar force-pushed the ci/blacksmith-runners branch from 7510416 to 8cd27c9 Compare February 23, 2026 14:19
@benvinegar benvinegar merged commit 0b7e79f into main Feb 23, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant